fix(output): suppress the pulumi-command AcceptEnv warning noise#237
Merged
Conversation
The remote provider tries to pass PULUMI_COMMAND_STDOUT/STDERR as SSH env vars on every command; OpenSSH's default AcceptEnv rejects them and the provider warns — four lines of non-actionable noise per resource, burying the real output. The vars only control the provider's own log forwarding (stdout/stderr capture as outputs is unaffected), and the alternative — loosening every host's sshd AcceptEnv — is a fleet config change for a cosmetic win. Suppressed in the Printer via an exact-prefix known-noise filter, kept deliberately short so it never becomes the place real warnings go to die. Real warnings still print (tested). Claude-Session: https://claude.ai/code/session_012HJAUraibyPthDne5QG2FW
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Drops the
Unable to set 'PULUMI_COMMAND_STDOUT/STDERR' … AcceptEnvwarnings from the deploy/preview stream via an exact-prefix known-noise filter in the Printer.Why
The pulumi-command remote provider emits this warning for every SSH command on every deploy — OpenSSH's default
AcceptEnvrejects the provider's log-forwarding env vars. It is non-actionable noise (command stdout/stderr capture as resource outputs is unaffected — the grant gating onmintCmd.Stdoutkeeps working), and it buried the real per-resource lines the new output exists to surface. Disabling it at the source isn't safe: the SDK'sLoggingoption would also drop the stdout capture grants depend on, and the alternative — addingAcceptEnv PULUMI_COMMAND_*to every host's sshd — is a fleet config change for a cosmetic win.The filter is deliberately exact-prefix and single-entry, with a comment warning against letting it grow into the place real warnings go to die. Real warnings still print (tested).
Merge Commit Message
fix(output): suppress the per-command pulumi-command AcceptEnv warning — non-actionable provider noise that buried the real deploy output; real warnings still print
https://claude.ai/code/session_012HJAUraibyPthDne5QG2FW